feat(address-review-pr): add --ci flag for non-interactive CI mode - #608
Merged
openshift-merge-bot[bot] merged 1 commit intoJul 7, 2026
Merged
openshift-merge-bot[bot] merged 1 commit into
openshift-merge-bot[bot] merged 1 commit into
Conversation
When --ci is passed, the skill runs fully non-interactively: - Dirty working tree: stash/discard instead of asking - Comment categorization: proceed without user confirmation - Ambiguous comments: make autonomous decisions instead of asking - Pre-push hook failures on pre-existing issues: push with --no-verify This prevents the agent from blocking on unanswered interactive prompts when running in CI pipelines. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
openshift-ci
Bot
added
the
approved
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (3)
WalkthroughThis PR documents a new ChangesSkill Documentation and Version Bump
Estimated code review effort: 1 (Trivial) | ~3 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (9 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
bryan-cox
approved these changes
Jul 7, 2026
Contributor
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bryan-cox, enxebre The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Merged
1 task
openshift-merge-bot Bot
pushed a commit
to openshift/release
that referenced
this pull request
Jul 10, 2026
…agent (#81721) The address-review-pr skill asks interactive questions (e.g., "Should I push with --no-verify?") that block the session in CI where there is no human to answer. The --ci flag makes the skill fully non-interactive. Depends on: openshift-eng/ai-helpers#608 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
SachinNinganure
pushed a commit
to SachinNinganure/release
that referenced
this pull request
Jul 20, 2026
…agent (openshift#81721) The address-review-pr skill asks interactive questions (e.g., "Should I push with --no-verify?") that block the session in CI where there is no human to answer. The --ci flag makes the skill fully non-interactive. Depends on: openshift-eng/ai-helpers#608 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
amogh-redhat
pushed a commit
to amogh-redhat/release
that referenced
this pull request
Aug 5, 2026
![openshift-prow[bot]](https://github.githubassets.com/images/gravatars/gravatar-user-420.png){kind=avatar}
…agent (openshift#81721) The address-review-pr skill asks interactive questions (e.g., "Should I push with --no-verify?") that block the session in CI where there is no human to answer. The --ci flag makes the skill fully non-interactive. Depends on: openshift-eng/ai-helpers#608 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
TimurMP
pushed a commit
to TimurMP/release
that referenced
this pull request
Sep 5, 2026
…agent (openshift#81721) The address-review-pr skill asks interactive questions (e.g., "Should I push with --no-verify?") that block the session in CI where there is no human to answer. The --ci flag makes the skill fully non-interactive. Depends on: openshift-eng/ai-helpers#608 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add
--ciflag to theaddress-review-prskill that makes it fully non-interactive.When
--ciis passed: NEVER ask interactive questions or wait for user input. Make autonomous decisions. When in doubt, proceed with the safest action.Why
The review-agent runs this skill in CI with no human on the other end. When the skill encounters ambiguity or pre-push hook failures, it asks for confirmation and the session ends without completing.
Example: build 2074442484376670208 — the agent wrote correct ratcheting tests for PR #8937, committed locally, but asked "Should I push with
--no-verify?" and the session ended with no push. 64 turns, $3.81 spent, work lost.Ref: CNTRLPLANE-3788
Test plan
--ciflag and verify it pushes without asking--ci) still prompts as before🤖 Generated with Claude Code
Summary by CodeRabbit
--cioption to the review-pr workflow docs for non-interactive automation.1.1.8in the marketplace and plugin metadata.